-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: 에셋 정리 및 패키지 관리 방법 변경 #694
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commit 2860d5d
에서 UIKit 을 추가했는데 이유가 있을까요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commit f1670cd
유지보수할 때 항상 고민되는 것이 화면에 대한 삭제인 것 같아요..! 만들어는 뒀는데 혹시 다시 쓰지는 않을까 하는거죵
그런데 현재 새롭게 카드 궁합 뷰로 대체하고 있으니 괜찮을 것 같아요!! 감사합니다~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SPM 으로 관리하는 방법 좋은 것 같아요! 그런데 SPM 에서 지원하지 않는 것들이 꽤나 많아서 선별해서 전환해야 할 것 같습니다...!! alamofire 만 우선 변경하실 건가요? 지금 커밋 내역에는 해당 기록이 없는데 확인할 수 있을까용?
@@ -8,7 +8,6 @@ | |||
// Configuration settings file format documentation can be found at: | |||
// https://help.apple.com/xcode/#/dev745c5c974 | |||
|
|||
#include "Pods/Target Support Files/Pods-NADA-iOS-forRelease/Pods-NADA-iOS-forRelease.debug.xcconfig" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
나머지 cocoapods 에서 제공하는 프레임워크는 필요할텐데 이렇게 삭제 해도 괜찮나용?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
더 나은 프로젝트를 위해서 노력해주셔서 감사합니다~~
이미지 에셋을 SVG 로 바꾸거나 폰트를 줄이는 방법 등 용량을 줄이기 위해 좋은 방법이 많네요..!
그리고 SPM 을 도입하는 것도 좋아요!!
고생하셨습니다..!! 코멘트 확인 부탁드려요~ 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
남아있던 코드들을 지우니 한층 프로젝트가 깔끔해진 것 같아요!
사실 오래된 프로젝트들의 문제점 중 하나가 레거시 코드가 많이 있다는 점인데,
이렇게 새로운게 생길때마다 프로젝트 정리를 한번씩 해주면 레거시 코드들이 남아있지 않아서 좋은 것 같네요~ 하나하나 고치느라 진짜 수고했담!
self.timesLeft = 600 - (interval.second ?? 0) | ||
print("✅ now: ", Date()) | ||
print("✅ activated Time: ", nearByUUIDResponse.activeTime?.toDate()) | ||
print("✅ timesleft: ", self.timesLeft) | ||
self.nearByTimeLabel.text = "" | ||
self.setCardActivationUI(with: nearByUUIDResponse.isActive, secondsLeft: self.timesLeft ?? 0) | ||
self.setCardActivationUI(with: nearByUUIDResponse.isActive, | ||
secondsLeft: self.timesLeft) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제가 요 부분을 개발했을때 굉장히 급하게 개발했던 것으로 기억합니다...
저 이니셜 값이 없어도 잘 돌아가던가요..?
🌴 PR 요약
🌱 작업한 브랜치
🌱 작업한 내용
제대 후 나다에서 처음 날리는 PR이네요.....설렙니다.
용량 줄이기 작업이 너무 무거워져서 쪼개려고 먼저 PR 올립니다!
1. 에셋 정리
2. 패키지 관리 방법 변경 (Cocoapods -> SPM)
이후에 어떤 작업?
📮 관련 이슈